-
-
Notifications
You must be signed in to change notification settings - Fork 281
fix(docs): correct heading hierarchy and Table of Contents in quickstart samples #780
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
Hey @Srinu346 can you please add video of your changes, i couldn't able to see any changes from the screenshot |
|
@Achanandhi-M sir, Here is a short video
Screen.Recording.2026-02-04.150402.mp4Sir as you can see here the right side table of content block is mixed of both docker compose steps and wsl steps. It is not alone in the Echo SQL Sample section, so where ever the issue is getting repeated i have modified them properly. It solved |
Achanandhi-M
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Srinu346, the changes look good. However, I noticed that no updates were made for the FastHTTP + Postgres setup. Could you please let me know why there are no changes there?
|
@Achanandhi-M sir, i already have a ongoing pr for FastHTTP + Postgres Setup so i avoided that in this pr. |
In that case, please don’t create a PR and request a review until the changes are fully completed. Kindly raise the PR once everything has been addressed. @Srinu346 |
|
Ok Sir, i will wait until the previous pr is completely merged or closed, then will look into this. |
What has changed?
This PR fixes incorrect heading hierarchies across multiple documentation pages that were causing the right-side Table of Contents to appear misaligned and inconsistent. Headings have been updated to follow a proper structure, and section titles were corrected so the Table of Contents accurately reflects the page content wherever the issue was repeated.
This PR resolves #3713
Type of change
How Has This Been Tested?
npm run buildto ensure the documentation builds successfully without errors.Checklist:
Screenshots